Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: fetch visualization always when caching (DHIS2-17509) #2986

Merged
merged 1 commit into from
Jun 6, 2024

Conversation

edoardo
Copy link
Member

@edoardo edoardo commented May 31, 2024

Implements DHIS2-17509


Key features

  1. fix regression with offline caching

Description

A previous fix for an item flashing issue caused the offline cache to lack the request for the visualization.
The fix looks at the recording state and triggers a fetch that can be recorded.
The affected version of Dashboard app is v100.2.2.


Screenshots

Before, the cache is missing the request for visualizations:

Screenshot 2024-05-31 at 15 59 21

After the cache has an entry for the visualizations request:

Screenshot 2024-05-31 at 15 57 02

@edoardo edoardo requested a review from jenniferarnesen May 31, 2024 14:07
@dhis2-bot
Copy link
Contributor

dhis2-bot commented May 31, 2024

🚀 Deployed on https://pr-2986--dhis2-dashboard.netlify.app

@dhis2-bot dhis2-bot temporarily deployed to netlify May 31, 2024 14:18 Inactive
@edoardo edoardo changed the title fix: fetch visualization always when caching fix: fetch visualization always when caching (DHIS2-17509) Jun 3, 2024
A previous fix for an item flashing issue caused the offline cache to
lack the request for the visualizations.
The fix looks at the recording state and triggers a fetch that can be
recorded.
@edoardo edoardo force-pushed the fix/visualization-offline-cache branch from 32bcaa1 to a480d10 Compare June 6, 2024 12:49
@dhis2-bot dhis2-bot temporarily deployed to netlify June 6, 2024 12:51 Inactive
@edoardo edoardo merged commit 8b3587e into dev Jun 6, 2024
19 checks passed
@edoardo edoardo deleted the fix/visualization-offline-cache branch June 6, 2024 12:56
@edoardo edoardo mentioned this pull request Jun 14, 2024
dhis2-bot added a commit that referenced this pull request Jun 17, 2024
## [100.2.3](v100.2.2...v100.2.3) (2024-06-17)

### Bug Fixes

* **translations:** sync translations from transifex (dev) ([7f44585](7f44585))
* **translations:** sync translations from transifex (dev) ([1c2d1b2](1c2d1b2))
* **translations:** sync translations from transifex (dev) ([ac3f1c7](ac3f1c7))
* **translations:** sync translations from transifex (dev) ([#3001](#3001)) ([018e07b](018e07b))
* fetch visualization always when caching (DHIS2-17509) ([#2986](#2986)) ([8b3587e](8b3587e))
@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 100.2.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants